Skip to content
This repository was archived by the owner on May 3, 2018. It is now read-only.

rename PartestTask to SbtPartestTask and bump versions #4

Merged
merged 2 commits into from
Oct 17, 2014

Conversation

gourlaysama
Copy link
Contributor

I've been looking into building Scala with SBT recently (poke @gkossakowski), and stumbled upon this.

PartestTask is a very unfortunate name for this, because it already
exists in scala-partest; there is:

  • scala.tools.partest.PartestTask extends org.apache.tools.ant.Task
    in scala-partest, for running partest with ant,
  • scala.tools.partest.PartestTask extends sbt.testing.Task for
    running partest with sbt.

Depending on the order scala-partest and scala-partest-interface are
added to the dependencies, SBT can get the wrong one and crash...


The second commit bumps version numbers (the last release was 0.4.0).

`PartestTask` is a very unfortunate name for this, because it already
exists in scala-partest; there is:

 - `scala.tools.partest.PartestTask extends org.apache.tools.ant.Task`
   in scala-partest, for running partest with ant
 - `scala.tools.partest.PartestTask extends sbt.testing.Task` for
   running partest with sbt

Depending on the order `scala-partest` and `scala-partest-interface` are
added to the dependencies, SBT can get the wrong one and crash...
@gkossakowski
Copy link
Contributor

LGTM

Good stuff!

gkossakowski added a commit that referenced this pull request Oct 17, 2014
rename PartestTask to SbtPartestTask and bump versions
@gkossakowski gkossakowski merged commit e9b3200 into scala:master Oct 17, 2014
@gourlaysama gourlaysama deleted the wip/partest-task branch October 18, 2014 19:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants